v0.9.0-beta.8: IBKR, snapshots, broker self-registration#87
Merged
luokerenx4 merged 4 commits intomasterfrom Mar 25, 2026
Merged
v0.9.0-beta.8: IBKR, snapshots, broker self-registration#87luokerenx4 merged 4 commits intomasterfrom
luokerenx4 merged 4 commits intomasterfrom
Conversation
createSnapshotService now accepts optional baseDir parameter. Service tests pass tmpdir and clean up in afterEach, preventing acc1/acc2 residue in the project's data directory. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
New README covers TWS market data channels (updatePortfolio vs reqMktData vs streaming), US equity trading sessions including overnight (Blue Ocean ATS), socket error handling design, and known limitations around price staleness after 20:00 ET. IbkrBroker methods (getAccount, getPositions, getQuote) annotated with data source, refresh behavior, and overnight coverage gaps. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Project Structure: rewrite to match domain/ + tool/ refactor (remove stale extension/ and openbb/ paths) - Features: add account snapshots, dynamic broker config UI, account enable/disable - Architecture: rename Extensions → Domain, add Snapshots node - Key Concepts: Extension → Domain + Tool layering - Configuration: add snapshot.json - Roadmap: mark all four v1 milestones complete Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
IbkrBrokeradapter (TWS/Gateway), callback→Promise bridge, conId-based contract resolution, graceful socket disconnect, completed orders syncconfigSchema+configFields+fromConfig; frontend dynamically renders config forms from/broker-typesAccountManagerowns full UTA lifecycle, runtime enable/disable toggle, connection verification before persistingdomain/+tool/refactor, all v1 roadmap items marked complete55 commits since last merge.
Test plan
pnpm test— all unit tests pass🤖 Generated with Claude Code